-
Notifications
You must be signed in to change notification settings - Fork 165
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
docs(1.7.0): add PSA for issue 9267 #974
Conversation
✅ Deploy Preview for longhornio ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
In general, LGTM. However, warnings could be multiple, so we better itemize them. I would like to have @longhorn/doc to suggest the formation. |
93823f3
to
11ba492
Compare
11ba492
to
41491f0
Compare
@jhkrug Can you help review it? Thank you. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I used a list to allow easier info scanning. Also, avoid using words like "soon" for better expectation management.
Signed-off-by: Derek Su <derek.su@suse.com>
9c9ea63
to
422b5cd
Compare
Which issue(s) this PR fixes:
Issue #
What this PR does / why we need it:
Special notes for your reviewer:
Additional documentation or context